Skip to content

Comments

Задание 1#694

Open
aoiaoki wants to merge 6 commits intoYandex-Practicum:mainfrom
aoiaoki:develop1
Open

Задание 1#694
aoiaoki wants to merge 6 commits intoYandex-Practicum:mainfrom
aoiaoki:develop1

Conversation

@aoiaoki
Copy link

@aoiaoki aoiaoki commented Feb 4, 2026

No description provided.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Необходимо исправить: эта директория лишняя в проекте. Необходимо убрать её из ветки и добавить в .gitignore в корне проекта

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Не исправлено

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Необходимо исправить: эта директория лишняя в проекте. Необходимо убрать её из ветки и добавить в .gitignore в корне проекта

from praktikum.bun import Bun


def test_bun_init():

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Нужно исправить здесь и далее: Для корректного запуска тестов необходимо код с шагами теста поместить в тестовый метод (нейминг начинается с test_), а метод - в тестовый класс (нейминг начинается с Test). Необходимо привести к такому формату все модули внутри пакета tests



def test_bun_init():
bun = Bun("black bun", 100)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Можно лучше здесь и далее: тестовые данные и ожидаемые ответы лучше не хранить в теле/модулях теста. Лучше вынести их в модуль data, так их будет легче поддерживать

Comment on lines 65 to 67
assert "(==== black bun ====)" in receipt
assert "= sauce hot sauce =" in receipt
assert "Price: 250" in receipt

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Нужно исправить: проверка через in в данном кейсе не будет надёжной, необходимо проверить полное соответствие

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants